Skip to content

[fix][CGS][entrance] fix sr task retry causing init_sql loading exception#5399

Merged
casionone merged 1 commit intoapache:masterfrom
v-kkhuang:master-init-sql-error-fix
Apr 2, 2026
Merged

[fix][CGS][entrance] fix sr task retry causing init_sql loading exception#5399
casionone merged 1 commit intoapache:masterfrom
v-kkhuang:master-init-sql-error-fix

Conversation

@v-kkhuang
Copy link
Copy Markdown
Contributor

What is the purpose of the change

Background/Problem:
When SR (Script Resource) tasks are retried, the init_sql loading process encounters exceptions. The current implementation does not properly handle the retry scenario, causing SQL initialization to fail during task retries.

Purpose of Change:
To address this problem, this PR fixes the init_sql loading logic to properly handle task retry scenarios, ensuring that SQL initialization works correctly during both initial execution and retries.

Value/Impact:
After the change, SR tasks can be retried without encountering init_sql loading exceptions, improving system reliability and task execution success rates.

Related issues/PRs

Related issues: close #5392
Related pr:none

Brief change log

  • Fix init_sql loading exception during SR task retry in ComputationExecutor
  • Update EntranceExecutionJob to handle retry scenario correctly
  • Update QueryPersistenceManager to support proper retry logic

Checklist

  • I have read the Contributing Guidelines on pull requests.
  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible
  • If this is a code change: I have written unit tests to fully verify the new behavior.

Copy link
Copy Markdown
Contributor

@casionone casionone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@casionone casionone merged commit 5fdf4a0 into apache:master Apr 2, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug][CGS][engineconn] fix sr task retry causing init_sql loading exception

2 participants